Skip to content

feat: cities add siuntio inkoo Refs: PL-217#1377

Open
mikkojamG wants to merge 1 commit into
mainfrom
feat/PL-217-cities-siuntio-inkoo
Open

feat: cities add siuntio inkoo Refs: PL-217#1377
mikkojamG wants to merge 1 commit into
mainfrom
feat/PL-217-cities-siuntio-inkoo

Conversation

@mikkojamG

@mikkojamG mikkojamG commented Jul 3, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

https://helsinkisolutionoffice.atlassian.net/browse/PL-217

Add cities Siuntio and Inkoo.

Azure: https://dev.azure.com/City-of-Helsinki/palvelukartta/_git/palvelukartta-pipelines/pullrequest/16377

Summary by CodeRabbit

  • New Features
    • Expanded the default city list to include Inkoo and Siuntio, making them available in app configuration and defaults.
  • Documentation
    • Updated setup and environment variable references to match the new city list.

@mikkojamG mikkojamG requested a review from a team as a code owner July 3, 2026 10:17
@coderabbitai

coderabbitai Bot commented Jul 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 1749c35d-5b6b-4a60-9526-f29f7cda6a83

📥 Commits

Reviewing files that changed from the base of the PR and between 1cc6c32 and f580945.

📒 Files selected for processing (4)
  • .env
  • .env.local.example
  • README.md
  • config/default.js
📝 Walkthrough

Walkthrough

This change adds two new cities, inkoo and siuntio, to the REACT_APP_CITIES configuration value across .env, .env.local.example, and config/default.js, and updates the corresponding default value documented in README.md.

Changes

City list expansion

Layer / File(s) Summary
Update city list across config and docs
.env, .env.local.example, config/default.js, README.md		 Adds inkoo and siuntio to the REACT_APP_CITIES comma-separated value in environment and config files, and updates the matching default value in the README documentation table.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Related PRs: None found.

Suggested labels: config, documentation

Suggested reviewers: None found.

🐰

Four little files, one small tweak,
Inkoo and Siuntio now on the streak,
Commas aligned, the city list grows,
Config, docs, and env — everyone knows!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly reflects the main change: adding Siuntio and Inkoo to the cities list.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/PL-217-cities-siuntio-inkoo

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

sonarqubecloud Bot commented Jul 3, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@azure-pipelines

azure-pipelines Bot commented Jul 3, 2026

Copy link
Copy Markdown

PALVELUKARTTA-UI branch is deployed to platta: https://palvelukartta-ui-pr1377.dev.hel.ninja 🚀🚀🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mikkojamG